Skip to content

fix: bound storage response and proxy idle times - #6

Merged
ehsanmsb merged 3 commits into
mainfrom
feature/storage-timeouts
Oct 29, 2026
Merged

ehsanmsb merged 3 commits into
mainfrom
feature/storage-timeouts

Conversation

@ehsanmsb

@ehsanmsb ehsanmsb commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Changes

  • Limit each S3 response-header wait to 5 seconds per attempt and each complete object transfer to 10 minutes by default; expose -s3-timeout and chart s3Timeout for tuning.
  • Return 504 when storage times out before headers; log a stream timeout if it occurs after headers.
  • Allow video transfers through Contour HTTPProxy with response: infinity, while closing streams idle for 60 seconds.
  • Document the distinction between time to first byte and total transfer time, and test timeouts before headers and during streaming.

Validation

  • go test -mod=readonly ./... passed.
  • go vet -mod=readonly ./... passed.
  • helm lint and helm template passed.

@ehsanmsb
ehsanmsb merged commit 8dc9a62 into main Oct 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant